V2 postgresql #597

ericgaspar · 2023-08-12T13:52:50Z

Transition to PostgreSQL database

Limezy · 2023-08-24T16:15:19Z

I have successfully updated my instance using that branch - thanks !

Limezy · 2023-08-25T04:53:15Z

Using the app control panel to change the PHP-FPM configuration destroys something and leaves to an error 502 !
EDIT : solved by changing php version to 8.2 inside /etc/php/8.2/fpm/pool.d/nextcloud.conf. I guess there is something slightly wrong within the control panel script

ericgaspar · 2023-10-29T17:32:03Z

!testme

yunohost-bot · 2023-10-29T17:32:06Z

Alrighty!

Limezy · 2023-10-31T07:20:01Z

By curiosity, is there a willing to transition to PostgreSQL for the official app branch or is this supposed to live as a parallel branch until PostgreSQL becomes the official recommended database system for Nextcloud ?

ericgaspar · 2023-10-31T07:24:49Z

where there's a will there's a way. More seriously this branch is more like a parallel universe for now.

ericgaspar · 2023-11-23T17:14:04Z

!testme

yunohost-bot · 2023-11-23T17:14:07Z

Fingers crossed!

Limezy · 2023-11-24T02:27:06Z

@ericgaspar thanks for your answer and also thanks for maintaining this parallel universe !

ericgaspar · 2023-12-12T22:45:20Z

!testme

yunohost-bot · 2023-12-12T22:45:23Z

Meow 🐈

ericgaspar · 2023-12-21T22:07:19Z

!testme

yunohost-bot · 2023-12-21T22:07:22Z

May the CI gods be with you!

ericgaspar · 2024-01-10T07:47:54Z

!testme

Limezy · 2024-01-10T07:56:29Z

The CI launching bot seems down

ericgaspar · 2024-01-10T13:13:13Z

scripts/upgrade

+then
+    ynh_script_progression --message="Migrating to PostgreSQL database..." --weight=3
+
+    ynh_backup_if_checksum_is_different --file="$install_dir/config/config.php"


Do we need this line?

ericgaspar · 2024-01-10T13:13:26Z

scripts/upgrade

+    ynh_mysql_remove_db --db_user=$db_user --db_name=$db_name
+    ynh_app_setting_delete --app=$app --key=mysqlpwd
+
+    ynh_store_file_checksum --file="${install_dir}/config/config.php"


...and this one?

alexAubin · 2024-01-10T18:04:53Z

scripts/upgrade

+mysql_db_pwd=$(ynh_app_setting_get --app=$app --key=mysqlpwd)
+
+if [ -n "$mysql_db_pwd" ]


I'm not sure we can really rely on the mysqlpwd ... It's supposed to be automatically deleted and renamed to db_pwd during the v1->v2 transition (so before running this script), cf https://github.com/YunoHost/yunohost/blob/dev/src/utils/resources.py#L1366-L1372

what about something like Mattermost https://github.com/YunoHost-Apps/mattermost_ynh/blob/52d95b9785edbe71e84fb9b99a26bec68544f123/scripts/upgrade#L40

alexAubin · 2024-01-10T18:10:34Z

scripts/upgrade

+    ynh_psql_test_if_first_run
+    ynh_psql_setup_db --db_user=$db_user --db_name=$db_name #--db_pwd=$(ynh_string_random)
+    #ynh_app_setting_set --app=$app --key=psqlpwd --value=$db_pwd


Now that we are in packaging v2, the DB should already be provisioned before the upgrade happens so eeeeh

scripts/upgrade

ericgaspar · 2024-01-10T18:33:53Z

!testme

yunohost-bot · 2024-01-10T18:33:56Z

🌻

alexAubin · 2024-01-10T22:31:50Z

+ sudo -u nextcloud php8.1 --define apc.enable_cli=1 occ --no-interaction --no-ansi db:convert-type --all-apps --clear-schema pgsql nextcloud 127.0.0.1 nextcloud --password=37f7b7c8d96539fff57a1df0 -n

An unhandled exception has been thrown:

Doctrine\DBAL\Exception: Failed to connect to the database: An exception occurred in the driver: could not find driver in /var/www/nextcloud/lib/private/DB/Connection.php:139

hmmmzgrmbl okay we have to make sure we have the appropriate phpX.Y-mysql/psql dependency installed during the migration, to be continued

ericgaspar · 2024-01-11T18:00:07Z

!testme

yunohost-bot · 2024-01-11T18:00:10Z

✌️

ericgaspar · 2024-01-11T20:50:11Z

!testme

yunohost-bot · 2024-01-11T20:50:14Z

Meow 🐈

Limezy · 2024-01-16T00:56:34Z

@alexAubin @ericgaspar do you have any idea about my case where the update fails from an already PostgreSQL migrated instance ? Many thanks !

ericgaspar · 2024-12-11T21:19:04Z

!testme

yunohost-bot · 2024-12-11T21:19:06Z

Living in the past, are we? 🐮👁️

yunohost-bot · 2024-12-11T21:19:06Z

😜

…cloud_ynh into v2_PostgreSQL

ericgaspar · 2024-12-12T12:10:31Z

!testme

yunohost-bot · 2024-12-12T12:10:34Z

🚀

yunohost-bot · 2024-12-12T12:10:34Z

Living in the past, are we? 🐮👁️

ericgaspar · 2024-12-16T17:42:42Z

!testme

yunohost-bot · 2024-12-16T17:42:47Z

Test it, book it, worm it, technologic. 🤖 🎶

yunohost-bot · 2024-12-16T17:42:47Z

Living in the past, are we? 🐮👁️

ericgaspar commented Jan 10, 2024

View reviewed changes

YunoHost-Apps deleted a comment from yunohost-bot Jan 10, 2024

YunoHost-Apps deleted a comment from alexAubin Jan 10, 2024

alexAubin reviewed Jan 10, 2024

View reviewed changes

scripts/upgrade Outdated Show resolved Hide resolved

ericgaspar mentioned this pull request Jan 11, 2024

Switch to PostgreSQL database? #107

Open

ericgaspar added 5 commits December 11, 2024 14:32

Update backup

393c62f

Update install

3681366

Merge branch 'testing' into v2_PostgreSQL

d70264b

Update nginx.conf

0065032

Update nginx.conf

22449fe

ericgaspar and others added 14 commits December 11, 2024 22:19

Merge branch 'testing' into v2_PostgreSQL

6c9727c

Update nginx.conf

f482f01

Update nginx.conf

cfa6ef0

Merge branch 'testing' into v2_PostgreSQL

53ded55

Update nginx.conf

9b21787

Update nginx.conf

43a05fe

Update nginx.conf

2405cc4

Merge branch 'testing' into v2_PostgreSQL

284ce68

Merge branch 'testing' into v2_PostgreSQL

28bf9b0

Update manifest.toml

c291b2e

Update manifest.toml

3aa9f9c

Auto-update READMEs

807afd3

fix

31bbb25

Merge branch 'v2_PostgreSQL' of https://github.com/YunoHost-Apps/next…

6912d19

…cloud_ynh into v2_PostgreSQL

ericgaspar added 2 commits December 14, 2024 18:28

Merge branch 'testing' into v2_PostgreSQL

e577054

cleaning

fb3b792

		mysql_db_pwd=$(ynh_app_setting_get --app=$app --key=mysqlpwd)

		if [ -n "$mysql_db_pwd" ]

V2 postgresql #597

Are you sure you want to change the base?

V2 postgresql #597

Conversation

ericgaspar commented Aug 12, 2023 • edited Loading

Limezy commented Aug 24, 2023

Limezy commented Aug 25, 2023 • edited Loading

ericgaspar commented Oct 29, 2023

yunohost-bot commented Oct 29, 2023

Limezy commented Oct 31, 2023

ericgaspar commented Oct 31, 2023

ericgaspar commented Nov 23, 2023

yunohost-bot commented Nov 23, 2023

Limezy commented Nov 24, 2023

ericgaspar commented Dec 12, 2023

yunohost-bot commented Dec 12, 2023

ericgaspar commented Dec 21, 2023

yunohost-bot commented Dec 21, 2023

ericgaspar commented Jan 10, 2024

Limezy commented Jan 10, 2024

ericgaspar Jan 10, 2024

Choose a reason for hiding this comment

ericgaspar Jan 10, 2024

Choose a reason for hiding this comment

alexAubin Jan 10, 2024

Choose a reason for hiding this comment

ericgaspar Jan 10, 2024 • edited Loading

Choose a reason for hiding this comment

alexAubin Jan 10, 2024

Choose a reason for hiding this comment

ericgaspar commented Jan 10, 2024

yunohost-bot commented Jan 10, 2024

alexAubin commented Jan 10, 2024 • edited Loading

ericgaspar commented Jan 11, 2024

yunohost-bot commented Jan 11, 2024

ericgaspar commented Jan 11, 2024

yunohost-bot commented Jan 11, 2024

Limezy commented Jan 16, 2024

ericgaspar commented Dec 11, 2024

yunohost-bot commented Dec 11, 2024

yunohost-bot commented Dec 11, 2024

ericgaspar commented Dec 12, 2024

yunohost-bot commented Dec 12, 2024

yunohost-bot commented Dec 12, 2024

ericgaspar commented Dec 16, 2024

yunohost-bot commented Dec 16, 2024

yunohost-bot commented Dec 16, 2024

ericgaspar commented Aug 12, 2023 •

edited

Loading

Limezy commented Aug 25, 2023 •

edited

Loading

ericgaspar Jan 10, 2024 •

edited

Loading

alexAubin commented Jan 10, 2024 •

edited

Loading